Redraw the shortcut section icons on one grid - #58
Merged
Conversation
The Touch glyph was hand-traced and filled, on a 390 242 320 400 viewBox, while every other icon was a 24-grid outline at stroke-width 2. It read as a dark blob beside its neighbours and turned to mush in the 14px chip nav. It is now drawn on the same grid, simplified to a single knuckle so the interior stays open at chip size. The rest follow from looking at the set rendered rather than at the markup: - Palette and commands used a wrench, which says settings or repair. The section is the drawing palette and the menu strips, so it is a palette. - Double-click was a mouse with two click arcs 1.4 units apart. They merged into a smear against the mouse body, and it left two near-identical mouse silhouettes in the chip nav. It is now a cursor, filled, because .chipnav a svg draws at opacity .68 and a thin outline disappeared there. - Mouse was 12x18 where the pen and keyboard fill 20, so it sat optically small. Now 14x20. - In every session had no icon, the only heading without one. The chip for the palette section is shortened to Palette. The chip track is capped at ~598px by .wrap.narrow and the fixed-width filter, independent of viewport, so the full label overflowed by ~110px at every desktop size. The heading keeps the full name. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Shortcuts page carried six section icons that had drifted apart. Reviewing them rendered, rather than as markup, turned up four real problems and one gap.
Touch was the outlier. It was hand-traced and filled, on a
390 242 320 400viewBox, while every other icon was a 24-grid outline atstroke-width="2". At heading size it read as a dark blob among light strokes; at 14px in the chip nav it was mush. It is redrawn on the same grid and simplified to a single knuckle, so the interior stays open at chip size — the three stacked finger humps were what made it feel packed.Palette and commands used a wrench, which says settings or repair. That section is the drawing palette and the menu strips, so it is now a palette.
Double-click did not survive small sizes. Its two click arcs sat 1.4 units apart on a 24 grid and butted against the mouse body; rendered, they merged into a smear. It also put two near-identical mouse silhouettes in the chip nav. It is now a cursor, and filled —
.chipnav a svgdraws atopacity: .68, so a thin outline was faint on top of thin and read as barely there.Mouse was optically small, 12×18 where the pen and keyboard fill 20 of the grid. Now 14×20, matching its neighbours' weight.
In every session had no icon at all, the only heading without one. It has a bolt now. This is the one piece not strictly required; it is easy to drop if you would rather the quick-reference block stay visually separate from the numbered groups.
Pen and Keyboard were already right and are untouched.
The chip label for the palette section is shortened to just Palette. The chip track is capped near 598px by
.wrap.narrowand the fixed-width filter input, independent of viewport, so the full label overflowed by roughly 110px at every desktop width — a wider monitor never helped. Six chips now measure 582px in that track. That leaves only about 16px of slack, so a seventh chip or a longer label will clip again; the levers if that happens are the 210px filter width or the 820pxnarrowcap instyles.css, neither touched here. The<h2>keeps the full "Palette and commands".All seven glyphs now sit on
0 0 24 24, centred at 12,12, atstroke-width="2"incurrentColor, so the--mutedcolour and dark mode continue to work. The chip nav and the headings are kept in sync. CRLF endings preserved to match the sibling pages.🤖 Generated with Claude Code